Skip to content

Remove any single member license sets from parsed licenses#420

Open
goneall wants to merge 1 commit into
masterfrom
issue419
Open

Remove any single member license sets from parsed licenses#420
goneall wants to merge 1 commit into
masterfrom
issue419

Conversation

@goneall
Copy link
Copy Markdown
Member

@goneall goneall commented May 14, 2026

Fixes #419

Fixes #419

Signed-off-by: Gary O'Neall <gary@sourceauditor.com>
@goneall
Copy link
Copy Markdown
Member Author

goneall commented May 20, 2026

@bact - could you do a quick review

@bact
Copy link
Copy Markdown
Collaborator

bact commented May 20, 2026

@goneall I'm looking at it now

import java.util.List;
import java.util.Objects;
import java.util.Optional;
import java.util.*;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure about Java coding style currently.
But Python linter will warn about any star import.

Not a blocker.

Copy link
Copy Markdown
Collaborator

@bact bact left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

License Expression Parse does not handle dis/conjunctive licenses with the same term

2 participants